Skip to content

Update de dependências e higiene do pacote.#1768

Open
pedrocava wants to merge 11 commits into
masterfrom
chore/package-update-and-clean-up
Open

Update de dependências e higiene do pacote.#1768
pedrocava wants to merge 11 commits into
masterfrom
chore/package-update-and-clean-up

Conversation

@pedrocava

Copy link
Copy Markdown
Collaborator

No description provided.

@rdahis

rdahis commented Jul 24, 2025

Copy link
Copy Markdown
Member

Cava back in action! Quando for para revisar é só marcar.

@aspeddro

aspeddro commented Sep 15, 2025

Copy link
Copy Markdown
Collaborator

Pode corrigir esses problemas https://cran.r-project.org/web/checks/check_results_basedosdados.html nesse PR @pedrocava?

Sabe o motivo dessa mensagem?
image

@pedrocava

pedrocava commented Sep 16, 2025

Copy link
Copy Markdown
Collaborator Author

Pode corrigir esses problemas https://cran.r-project.org/web/checks/check_results_basedosdados.html nesse PR @pedrocava?

Sabe o motivo essa mensagem? image

Consigo sim, o CRAN me contatou pedindo para resolver também. É só um probleminha de formatação da URL que deixamos nas docs.

@pedrocava pedrocava marked this pull request as ready for review September 16, 2025 02:50
Comment thread r-package/R/http-client.R
endpoint,
query = list()) {

base_url <- "https://basedosdados.org/api/3/action/bd_"

@aspeddro aspeddro Sep 16, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esse API não exite, certo @rdahis? Usamos https://backend.basedosdados.org/api/v1/graphql

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Certo! Essa era a api do ckan

@folhesgabriel

folhesgabriel commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

Pode corrigir esses problemas https://cran.r-project.org/web/checks/check_results_basedosdados.html nesse PR @pedrocava?
Sabe o motivo essa mensagem? image

Consigo sim, o CRAN me contatou pedindo para resolver também. É só um probleminha de formatação da URL que deixamos nas docs.

Valeu @pedrocava
@pedrocava Vc que tem feito o processo de submissão do pacote ao CRAN? Dada a ermegência, pensei com @aspeddro em fazer a submissão apenas com o problema relatado pelo CRAN e, na sequência, faremos um mapeamento de melhorias (atualizar dependências; atualizar métodos de consulta aos metadados...) para implementar com mais calma

@pedrocava

Copy link
Copy Markdown
Collaborator Author

Dá para submeter hoje, as-is, e subimos outra versão depois resolvendo essas funções que batem na API de metadados. A gente mantém o PR aberto enquanto isso. Todo mundo de acordo?

@aspeddro

Copy link
Copy Markdown
Collaborator

Concordo. Mantem esse aberto e abre um PR com as correções do cran 6fbc7b3

@folhesgabriel folhesgabriel marked this pull request as draft September 16, 2025 17:25
@pedrocava

Copy link
Copy Markdown
Collaborator Author

Na verdade, a ideia era manter esse PR aqui mesmo. Ele já tem mudanças úteis e mudar apenas o que o CRAN pediu não passa no CMD check. Fechado, galera?

@folhesgabriel

Copy link
Copy Markdown
Contributor

Combinado!

@folhesgabriel folhesgabriel marked this pull request as ready for review September 16, 2025 19:49
@pedrocava

Copy link
Copy Markdown
Collaborator Author
Dear maintainer,

thanks, package basedosdados_0.2.3.tar.gz is on its way to CRAN.

Best regards,
CRAN teams' auto-check service
Flavor: r-devel-linux-x86_64-debian-gcc, r-devel-windows-x86_64
Check: *, Result: OK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants